Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure logs use the correct timezone in CLI and Web contexts #1087

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

M0rgan01
Copy link
Contributor

@M0rgan01 M0rgan01 commented Dec 16, 2024

Questions Answers
Description? see below
Type? feature
BC breaks? no
Deprecations? no
Fixed ticket? -
Sponsor company -
How to test? See below

This PR updates the logging behavior to ensure timestamps are correctly aligned with the execution context:

  • CLI Context: Logs will use the system's timezone.
  • Web Context: Logs will use the timezone configured in the PrestaShop settings.

@M0rgan01 M0rgan01 added this to the 7.0.0 milestone Dec 16, 2024
nicosomb
nicosomb previously approved these changes Dec 16, 2024
Quetzacoalt91
Quetzacoalt91 previously approved these changes Dec 17, 2024
@AureRita AureRita self-assigned this Dec 17, 2024
@M0rgan01 M0rgan01 added the Blocked Status: The issue is blocked by another task label Dec 19, 2024
@M0rgan01 M0rgan01 dismissed stale reviews from Quetzacoalt91 and nicosomb via 2370653 February 3, 2025 09:13
@M0rgan01 M0rgan01 force-pushed the remove-force-timezone branch from dfcf3a5 to 2370653 Compare February 3, 2025 09:13
@M0rgan01 M0rgan01 changed the title [FIX] Withdrawal of the forced timezone in Paris Ensure logs use the correct timezone in CLI and Web contexts Feb 3, 2025
@M0rgan01 M0rgan01 removed the Blocked Status: The issue is blocked by another task label Feb 3, 2025
@M0rgan01 M0rgan01 force-pushed the remove-force-timezone branch 4 times, most recently from c25206b to ecca142 Compare February 4, 2025 09:36
@M0rgan01 M0rgan01 force-pushed the remove-force-timezone branch from ecca142 to 4adb0ba Compare February 4, 2025 10:09
Copy link

sonarqubecloud bot commented Feb 4, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for review
Development

Successfully merging this pull request may close these issues.

4 participants